Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Remove unneeded toolchain_resolution_debug flags from presubmit. #245

Merged
merged 1 commit into from
Apr 12, 2021
Merged

Remove unneeded toolchain_resolution_debug flags from presubmit. #245

merged 1 commit into from
Apr 12, 2021

Conversation

katre
Copy link
Contributor

@katre katre commented Apr 12, 2021

Fixes #243.

@google-cla google-cla bot added the cla: yes label Apr 12, 2021
@katre katre changed the title [WIP] Remove unneeded toolchain_resolution_debug flags from presubmit. Remove unneeded toolchain_resolution_debug flags from presubmit. Apr 12, 2021
@katre katre requested a review from tomaszstrejczek April 12, 2021 12:59
@katre katre marked this pull request as ready for review April 12, 2021 12:59
@katre katre requested a review from zaphar as a code owner April 12, 2021 12:59
@tomaszstrejczek tomaszstrejczek merged commit 28bfd0b into bazel-contrib:master Apr 12, 2021
@katre katre deleted the fix-toolchain-resolution-debug branch April 12, 2021 18:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@tomaszstrejczek tomaszstrejczek Awaiting requested review from tomaszstrejczek

@zaphar zaphar Awaiting requested review from zaphar

Assignees
No one assigned
Labels
cla: yes
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

rules_dotnet broken in Bazel downstream due to recent change to --toolchain_resolution_debug
2 participants
@katre @tomaszstrejczek